Skip to content

[8.x](backport #41845) feat: drop init.d support and go-daemon binaries#42719

Merged
kruskall merged 7 commits into8.xfrom
mergify/bp/8.x/pr-41845
Mar 24, 2025
Merged

[8.x](backport #41845) feat: drop init.d support and go-daemon binaries#42719
kruskall merged 7 commits into8.xfrom
mergify/bp/8.x/pr-41845

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Feb 14, 2025

Proposed commit message

remove init.d scripts
drop go-daemon dep and daemon binaries
cleanup mage task

init.d should not be needed on distros we support
According to go-daemon readme: "Go daemon is not needed on systems with systemd."

See https://www.elastic.co/support/matrix#matrix_os

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Disruptive User Impact

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

This is an automatic backport of pull request #41845 done by [Mergify](https://mergify.com).

@kruskall @mergify
feat: drop init.d support and go-daemon binaries (#41845)
94d11a9 
* feat: drop init.d support and go-daemon binaries

remove init.d scripts
drop go-daemon dep and daemon binaries
cleanup mage tasks

* lint: fix linter issues

* docs: remove reference to init.d scripts

(cherry picked from commit 5fbf359)
@mergify mergify bot added the backport label Feb 14, 2025
@mergify mergify bot requested review from a team as code owners February 14, 2025 16:40
@mergify mergify bot requested review from belimawr and rdner and removed request for a team February 14, 2025 16:40
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Feb 14, 2025
@github-actions github-actions bot added enhancement Team:obs-ds-hosted-services Label for the Observability Hosted Services team Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team Team:Security-Linux Platform Linux Platform Team in Security Solution Team:Security-Windows Platform Windows Platform Team in Security Solution Team:Security-Deployment and Devices Deployment and Devices Team in Security Solution labels Feb 14, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Feb 14, 2025

Pinging @elastic/obs-ds-hosted-services (Team:obs-ds-hosted-services)

@elasticmachine
Copy link
Contributor

elasticmachine commented Feb 14, 2025

Pinging @elastic/sec-linux-platform (Team:Security-Linux Platform)

@elasticmachine
Copy link
Contributor

elasticmachine commented Feb 14, 2025

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@elasticmachine
Copy link
Contributor

elasticmachine commented Feb 14, 2025

Pinging @elastic/sec-windows-platform (Team:Security-Windows Platform)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Feb 14, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Feb 14, 2025

Pinging @elastic/sec-deployment-and-devices (Team:Security-Deployment and Devices)

@mergify
Copy link
Contributor Author

mergify bot commented Feb 17, 2025

This pull request has not been merged yet. Could you please review and merge it @kruskall? 🙏

@mergify
Copy link
Contributor Author

mergify bot commented Feb 24, 2025

This pull request has not been merged yet. Could you please review and merge it @kruskall? 🙏

@mergify
Copy link
Contributor Author

mergify bot commented Mar 3, 2025

This pull request has not been merged yet. Could you please review and merge it @kruskall? 🙏

@rdner rdner removed their request for review March 7, 2025 12:02
@mergify
Copy link
Contributor Author

mergify bot commented Mar 7, 2025

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b mergify/bp/8.x/pr-41845 upstream/mergify/bp/8.x/pr-41845
git merge upstream/8.x
git push upstream mergify/bp/8.x/pr-41845

@mergify
Copy link
Contributor Author

mergify bot commented Mar 10, 2025

This pull request has not been merged yet. Could you please review and merge it @kruskall? 🙏

@mergify
Copy link
Contributor Author

mergify bot commented Mar 17, 2025

This pull request has not been merged yet. Could you please review and merge it @kruskall? 🙏

@mergify
Copy link
Contributor Author

mergify bot commented Mar 24, 2025

This pull request has not been merged yet. Could you please review and merge it @kruskall? 🙏

@kruskall kruskall merged commit 5942a2c into 8.x Mar 24, 2025
190 checks passed
@kruskall kruskall deleted the mergify/bp/8.x/pr-41845 branch March 24, 2025 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@belimawr belimawr belimawr approved these changes

@andrewkroh andrewkroh andrewkroh approved these changes

+1 more reviewer

@kruskall kruskall kruskall approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@kruskall kruskall

Labels

backport enhancement Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team Team:obs-ds-hosted-services Label for the Observability Hosted Services team Team:Security-Deployment and Devices Deployment and Devices Team in Security Solution Team:Security-Linux Platform Linux Platform Team in Security Solution Team:Security-Windows Platform Windows Platform Team in Security Solution

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@elasticmachine @belimawr @andrewkroh @kruskall